-
Notifications
You must be signed in to change notification settings - Fork 42
Include tls-manager-plugin in-tree #239
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
mgaffigan
wants to merge
363
commits into
OpenIntegrationEngine:main
Choose a base branch
from
mgaffigan:feature/add-tls
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When the isTlsManagerEnabled property is set to `true`, a TLS connection test is performed. If it is false, the default non-tls connection test is performed instead.
This change introduces separate testing functions for each connector, thus reducing the need to acoommodate different approaches.
…tificateDialog component for consistent confirmation across multiple dialogs. Integrate it into EditAliasDialog, ImportCertificateChainDialogContent, ImportFromUrlDialogContent, ImportPrivateCertificateDialog, and ImportTrustedCertificateDialog, ensuring users are informed about existing certificates before replacement. Update hooks to manage existing certificate information for improved user experience.
…: Simplify state management by removing unnecessary refs and effects for measuring content height. Implement expand/collapse functionality based on item count thresholds for improved user experience. Update error messages in useCertificateImport for clearer validation feedback. Add .pem to .gitignore for better file management.
Remove Git hash from Plugin Version identifier
…plugin.git as subtree Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Opening for discussion. Incorporates https://github.com/NovaMap-Health/tls-manager-plugin via
git subtreeand an ant build shim.Open issues: